Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

misc/pyjail-1 #67

Merged
merged 3 commits into from
Jun 3, 2024
Merged

misc/pyjail-1 #67

merged 3 commits into from
Jun 3, 2024

Conversation

Jack-Crowley
Copy link
Contributor

No description provided.

@Jack-Crowley Jack-Crowley added misc Miscellaneous challenge needs-deploy Challenge needs Dockerfile or deployment config labels Jun 3, 2024
pyjail-1/solve.md Outdated Show resolved Hide resolved
@mud-ali mud-ali added the needs-review Feel free to review this challenge label Jun 3, 2024
@mud-ali
Copy link
Member

mud-ali commented Jun 3, 2024

lgtm, i would just suggest combining the two python files (esp since theyve desynced in your last commit 👀 ) and read the flag from a text file instead of manually redacting

@mud-ali mud-ali added approved Challenge approved and removed needs-deploy Challenge needs Dockerfile or deployment config needs-review Feel free to review this challenge labels Jun 3, 2024
@mud-ali
Copy link
Member

mud-ali commented Jun 3, 2024

lgtm, i would just suggest combining the two python files (esp since theyve desynced in your last commit 👀 ) and read the flag from a text file instead of manually redacting

...which, i fixed btw

@mud-ali mud-ali merged commit af5fc5e into main Jun 3, 2024
1 check passed
@glacialcascade glacialcascade deleted the misc/pyjail-1 branch June 10, 2024 23:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Challenge approved misc Miscellaneous challenge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants